[Paged File, I/O System] 초코칩(권기호) 미션 제출합니다. #3
Conversation
| import java.util.ArrayList; | ||
| import java.util.List; | ||
|
|
||
| public class Page implements Serializable { |
There was a problem hiding this comment.
Serializable을 이용해서 페이지를 구현해주셨군요.
추후에 페이지 구조가 변경되면 이미 저장된 데이터를 역직렬화 할 수 있을까요?
우리가 만든 DBMS가 완성되어 사용자가 테이블을 만들어 데이터를 추가한 상황에서 페이지 클래스의 필드가 추가되거나 삭제, 수정되면 기존 데이터를 역직렬화 할 수 있나요?
| @@ -0,0 +1,16 @@ | |||
| package database.storageEngine.page; | |||
|
|
|||
| public class PageFactory { | |||
There was a problem hiding this comment.
이부분은 Page 클래스에 정적 팩토리 메서드로 만들 수 있는데 따로 팩토리 클래스를 만든 이유가 궁금합니다. 초코칩이 생각하는 분리의 기준이 있나요? 궁금해요!
| this.pageHeader = new PageHeader(pageType); | ||
| this.userStorageRecords = new ArrayList<>(); | ||
| this.pageDirectory = new PageDirectory(); | ||
| this.freeSpace = PAGE_SIZE - (this.fileHeader.getHeaderSize() + this.pageHeader.getHeaderSize()); |
There was a problem hiding this comment.
freeSpace를 계산할때 pageDirectory의 크기는 고려되지 않고 있는데요. freeSpace가 레코드로 가득 찬경우 pageDirectory의 크기 때문에 PAGE_SIZE가 초과되는 문제는 없을까요?
|
|
||
| private static final int PAGE_SIZE = 16 * 1024; | ||
| private static final String DIRECTORY_PATH = "disk"; | ||
| private static final String FILE_EXTENSION = ".ibd"; |
There was a problem hiding this comment.
추후에 테이블의 스키마를 저장한다면 .ibd 외에 다른 확장자 파일을 만들 수도 있을것 같아요. 그때 이 클래스를 재활용 할 수 있을것 같은데 파일확장자는 상수가 아닌 인자로 받는건 어떤가요?
| return Optional.of(page); | ||
| } | ||
| } catch (IOException | ClassNotFoundException e) { | ||
| return Optional.empty(); |
There was a problem hiding this comment.
IOException에 대해서 Optional.empty()을 리턴해도 문제가 없을까요? 파일을 읽는 과정에서 문제가 발생했을때 페이지가 없다고 하면 추후에 버그가 발생했을때 잡기 어려운 코드가 될 수 있을것 같아요.
src/main/java/database/storageEngine/bufferpool/pageReplacementStrategies/LFUStrategy.java
Show resolved
Hide resolved
| private static final int HEADER_SIZE = 38; | ||
|
|
||
| private final long pageNumber; | ||
| private final long checksum; |
|
|
||
| private static final int HEADER_SIZE = 38; | ||
|
|
||
| private final long pageNumber; |
There was a problem hiding this comment.
저는 페이지 번호 자료형을 int로 사용했는데 long 타입을 사용할만큼 파일 크기가 커질지..?
적절한 자료형을 잘 모르겠습니다 어떻게 생각하시나요
|
코드 잘 봤습니다! 앞으로도 화이팅입니다! |
페이지는 어떻게 구성되나요?
아래 그림의 페이지의 구조를 참고하여 구현했습니다.
FileTrailer는 Page의 유효성을 검사하는 역할인데, 현재 단계에서는 굳이 필요성을 못느껴 구현하지 않았습니다.
페이지 교체 알고리즘은 어떤 것으로 결정하고, 왜 결정했나요?
FIFO는 성능이 안좋다고 알려져 있어, 당장 구현하기 좋았던 LRU 알고리즘을 적용했습니다. 어떤 알고리즘이 좋은지 몰라서, 알고리즘이 변경될 가능성이 있다고 판단했습니다. 따라서 전략 패턴을 적용해서 알고리즘을 교체할수 있게 구현했습니다.